Ghost context provider #3506
Ghost context provider #3506
Conversation
…nd recentlyEdited
|
markijbema
left a comment
markijbema
left a comment
There was a problem hiding this comment.
I'd expect there to be code to register/deregister the ContextRetrievalService. So either it does it itself, but in that case there is some missing teardown code, or it doesnt, in which case it will always give an empty result.
Either way, even with that this does look way simpler than expected! Very cool
…g/kilocode into beatlevic/ghost-context-provider
src/services/ghost/classic-auto-complete/GhostContextProvider.ts
Outdated
Show resolved
Hide resolved
src/services/ghost/classic-auto-complete/GhostContextProvider.ts
Outdated
Show resolved
Hide resolved
| const filepathUri = filepath.startsWith("file://") ? filepath : vscode.Uri.file(filepath).toString() | ||
|
|
||
| // Initialize import definitions cache | ||
| await this.contextService.initializeForFile(filepathUri) |
There was a problem hiding this comment.
this doesnt seem thread safe; we can have multiple requests in flight, and then we initialize it for one file but by the time we create the snippetPayload it might have already been changed to another file
src/services/ghost/classic-auto-complete/GhostContextProvider.ts
Outdated
Show resolved
Hide resolved
| // Convert all snippet filepaths to URIs | ||
| const snippetsWithUris = filteredSnippets.map((snippet: any) => ({ | ||
| ...snippet, | ||
| filepath: snippet.filepath?.startsWith("file://") |
There was a problem hiding this comment.
again, why do we have this in two different formats; is this a real thing or just llm slop?
Added ghost context provider that retrieves all relevant context in a comment wrapped style.
Adds the comment wrapped code to the QUERY prompt.